Skip to content

fix: update devcontainer's docker image to match github#1323

Closed
vegerot wants to merge 1 commit into
facebook:mainfrom
vegerot:pr1323
Closed

fix: update devcontainer's docker image to match github#1323
vegerot wants to merge 1 commit into
facebook:mainfrom
vegerot:pr1323

Conversation

@vegerot

@vegerot vegerot commented May 27, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 27, 2026 20:30
@meta-cla meta-cla Bot added the CLA Signed label May 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the VS Code devcontainer configuration to build from a different Dockerfile (manylinux 2.34), likely to align the local dev environment with a new base image/toolchain.

Changes:

  • Switched the devcontainer build Dockerfile from sapling-cli-ubuntu-22.04.Dockerfile to manylinux_2_34.Dockerfile.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -2,7 +2,7 @@
"name": "Sapling",
"build": {
"context": "..",
@vegerot vegerot closed this May 27, 2026
@meta-codesync

meta-codesync Bot commented May 27, 2026

Copy link
Copy Markdown

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D106554635. (Because this pull request was imported automatically, there will not be any future comments.)

meta-codesync Bot pushed a commit that referenced this pull request May 28, 2026
Summary: Pull Request resolved: #1323

Reviewed By: quark-zju

Differential Revision: D106554635

fbshipit-source-id: ad5413c8a1fe45624b6ec3270409e34ca1b4c802
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants